refactored CheckServerJobFactory to take a raw account pointer. Also removed the parent arg to the factory as it's fairly useless
adjusted the parenting of the one-off nam created in the factory to be sure it is parented by the job, so it is cleaned up along with the job.
minimal update to oauth to deal with the new factory interface

…ss existing.

also cleaned up some additional accessors in the abstract network job

…count, and got rid of account member for DiscoverySingleDirectoryJob as it was never used

… state to the spacesmanager as that was all that was needed

eliminated more accessors to get the account, replaced more accountptr with just account*.
identified cases where the account dep can be removed completely with correct refactoring to make jobs only run in controllers/managers (where it's reasonable).
removed no longer used deprecated function from SpacesManager

…actor/DC-150

…d setAccountStatus function. Now the account status handling is much safer and clearer.

…of a qset of accountptr instances I changed it to a hash of uuid -> QPointer<account>

I think we can actually replace this with a qset of defaultSyncRoot since the roots are unique to the account, and that is all the class needs to function, but don't want to get too far off track with this pr so do that another time

…asses now)

…used directly by subclasses

updated fetchServerSettings and discovery classes

…ntly vfs tests don't pass - I am pushing this so I can pull the vfs changes

…fs - still need to turn it into a qpointer and check for nulls but it works now (local tests passed too) so should be no problem.

… gone.

also did some basic cleanup on some includes as well as updating to #pragma once where I ran into it